Skip to content

[16.0][IMP] base_geoengine: Fix Vector Layer related Error#417

Open
anusriNPS wants to merge 1 commit intoOCA:16.0from
PyTech-SRL:16.0-fix-geoview
Open

[16.0][IMP] base_geoengine: Fix Vector Layer related Error#417
anusriNPS wants to merge 1 commit intoOCA:16.0from
PyTech-SRL:16.0-fix-geoview

Conversation

@anusriNPS
Copy link
Contributor

@anusriNPS anusriNPS commented Oct 8, 2025

Notifying user to define selected attribute_field_id of supported type from vector layer as part of geoengine view xml defintion which avoids below observed JS error.

image

@anusriNPS
Copy link
Contributor Author

anusriNPS commented Oct 8, 2025

Notification displayed when supported type of attribute_field_id is not available as part of geoengine view definition:
Screenshot from 2025-10-08 16-52-26

@anusriNPS anusriNPS changed the title [16.0][IMP] base_geoengine: Fix Vector Layer related Errors [16.0][IMP] base_geoengine: Fix Vector Layer related Error Oct 8, 2025
    Notifying user to define selected attribute_field_id of
supported type from vector layer as part of geoengine view
xml defintion which avoids observed JS error.
Copy link
Contributor

@HekkiMelody HekkiMelody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review, LGTM

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 22, 2026
@HekkiMelody
Copy link
Contributor

@OCA/geospatial-maintainers Could you please take a look? Thanks!

Copy link
Member

@yvaucher yvaucher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM code review only

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants